-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed test_write_report() log path issue #954
Conversation
Codecov Report
@@ Coverage Diff @@
## master #954 +/- ##
==========================================
- Coverage 87.34% 87.33% -0.02%
==========================================
Files 65 65
Lines 8085 8099 +14
==========================================
+ Hits 7062 7073 +11
- Misses 1023 1026 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Please rebase on current master, there was a fix to stalling. |
validate_bids ongoing
… such for testing.
Hi @jwodder — I understand this is somewhat urgent and @yarikoptic might not be with us next week. Can you have a look at this and let me know if it's ok to merge even with the slight test coverage decrease? The log files get dropped either under pytest's |
🚀 PR was released in |
Closes #948
Fix for another function is ongoing, hopefully done within the hour.